Fixed Costa Rica's countryData to fix a couple of issues.#695
Fixed Costa Rica's countryData to fix a couple of issues.#695kaio-donadelli wants to merge 1 commit into
Conversation
|
Hi! I'm VTEX IO CI/CD Bot and I'll be helping you to publish your app! 🤖 Please select which version do you want to release:
And then you just need to merge your PR when you are ready! There is no need to create a release commit/tag.
|
|
0 New Issues
0 Fixed Issues
0 Accepted Issues
No data about coverage (74.00% Estimated after merge)
fdaciuk
left a comment
fdaciuk
left a comment
There was a problem hiding this comment.
Sem pontos bloqueantes. Duas sugestões inline abaixo.
|
|
||
| ### Fixed | ||
|
|
||
| - Costa Rica's countryData to fix a couple of issues. |
There was a problem hiding this comment.
Seria mais descritivo especificar quais problemas foram corrigidos, ex.: "Fixed typo in district name San Lorenzo (San Ramón, Alajuela) and incorrect postal code for Aserrí district".
| }, | ||
| "Aserrí": { | ||
| "Aserrí": "10504", | ||
| "Aserrí": "10601", |
There was a problem hiding this comment.
Incluir na descrição do PR uma referência ao CEP oficial (ex.: site dos Correios da Costa Rica) facilitaria auditorias futuras desse arquivo.
3 participants
What is the purpose of this pull request?
This fixes an incorrect postal code and a typo in Costa Rica country data.
Types of changes